-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changed the endpoint variable to be dependent on GKE cluster state #214
Changed the endpoint variable to be dependent on GKE cluster state #214
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please be sure to regenerate docs (make generate_docs
). I'm not sure why your commit is removing docs right now.
@morgante My commit was removing the inputs due to me not running Not sure why the lint tests are failing. |
@kevensen Can you unstage all README changes? Something is funky with your |
When I run I've backed out the changes to the README.md's. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linting failures look unrelated, once CI comes back green LGTM.
@morgante Thank you! |
…updates Changed the endpoint variable to be dependent on GKE cluster state
Addresses #212